Skip to content

feat: add Novita OpenAI-compatible provider#1266

Closed
Alex-wuhu wants to merge 1 commit intoruvnet:mainfrom
Alex-wuhu:feat/novita-openai-provider
Closed

feat: add Novita OpenAI-compatible provider#1266
Alex-wuhu wants to merge 1 commit intoruvnet:mainfrom
Alex-wuhu:feat/novita-openai-provider

Conversation

@Alex-wuhu
Copy link

@Alex-wuhu Alex-wuhu commented Mar 2, 2026

Summary

  • add novita as a first-class provider type in providers package
  • introduce NovitaProvider that reuses OpenAI provider behavior with default endpoint https://api.novita.ai/openai
  • wire Novita into provider exports and provider manager switch
  • update OpenAI provider internals to honor runtime provider label (openai or novita)
  • add unit test for Novita endpoint routing and response provider label

Verification

  • attempted: npm test -- src/tests/novita-provider.test.ts
  • result: failed with vitest not found because test dependency is not installed in this workspace

@Alex-wuhu Alex-wuhu marked this pull request as ready for review March 3, 2026 03:18
@Alex-wuhu
Copy link
Author

CI still failing after 3 fix attempts. Leaving for manual review.

@Alex-wuhu Alex-wuhu closed this Mar 3, 2026
@Alex-wuhu Alex-wuhu reopened this Mar 3, 2026
@Alex-wuhu
Copy link
Author

关闭此 PR:修改了 openai-provider.ts 核心逻辑,违反了不修改原有 provider 的原则。正确实现见 #1237

@Alex-wuhu Alex-wuhu closed this Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant